-
Notifications
You must be signed in to change notification settings - Fork 781
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
nightly #1906
base: develop
Are you sure you want to change the base?
nightly #1906
Conversation
maybe fix the CI?
OK, CI's seem to be fixed. The trigger is "on push" on the "develop" branch, so it will catch PR merges (and direct commits, if you ever do that). One workflow publishes linux arm64, the other publishes x86_64 wheels, using test.pypi. I "pip installed" from test.pypi on my raspberry pi and all my tests pass. :-) To turn this on for borglab, you'll have to make a pypi project for it (e.g. "gtsam-develop" or something), and change the name in the workflow to match, and add pypi "trusted publishers" matching the workflow names. I can do some of those parts if you like. |
That would be awesome ! |
Oh one more thing, i think there's got to be some way to replace the zillion commits mentioned here with one squashed commit, but I'm not enough of a git expert to know how to do that. Does anyone know? Otherwise, I could just manually copy the changes to a fresh branch. |
When merging you could do a "squash and merge" instead of a regular merge. There should be a drop-down next to the merge button. |
oh! that sounds easy. i don't have that button since i'm not a repo member, maybe that's something you-all could do? |
Let’s cross that bridge when we merge. Is this fully ready for review at this point? |
@truher this is very useful! One concern: I tried using one of these nightly builds but could not, because it was not supported on the system (which does support the pip released gtsam). I think this might be from the I could not spot any other mismatch in the requirements vs my setup. |
Hello all, sorry for not jumping in earlier, deadline is very tight. I don't know if @truher has looked at the Also AFAIK, https://github.com/borglab/gtsam-manylinux-build/tree/master already can build from the |
No, not at all, that's just what comes with my desktop, Ubuntu 22. It should be possible to use an older tag. |
I'm sorry to be a burden. There's no hurry with this change (since it's working for me, I'm not blocked on integrating it).
I was dimly aware of that, yes, but I was trying to do the simplest possible thing, and it didn't seem simple to figure out how to get docker to work with qemu or some cross-compiler solution (e.g. I experimented with python's crossenv, but that didn't seem simple either). My binary target is arm64 for Raspberry Pi. (I also have to admit that I'm not much of a fan of docker in general, and the Tower of Babel that python build tools have become also scares me.)
Would that work for ARM? |
There is probably no need for that, since you can just use an aarch64 runner on GitHub I guess? Unless you want to support armv7 which probably you don't.
Yes, quoting
|
Ah, my bad - GitHub actions does not support ARM64 runners yet. So you may need to use docker's |
as I understand it, docker is not an emulator, so you can't run an arm64 image within an amd64 host. isn't that true? |
https://www.stereolabs.com/docs/docker/building-arm-container-on-x86 |
yeah, qemu. that's pretty much what I'm doing, just without docker. |
You can use LXC or chroot if you don't want to use Docker, but you will need to have an older glibc anyways. |
No description provided.